CreateLeaveRequestCommandHandler.execute - Code Metrics - Inspection of "Merge pull request #436 from fairnesscoop/feat-mm-..." - fairnesscoop/permacoop - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 26b471...d6088a )

by Mathieu

created 2024-06-28 09:18 UTC

CreateLeaveRequestCommandHandler.execute B

↳ Parent: CreateLeaveRequestCommandHandler

Complexity

Conditions

Size

Total Lines	71
Code Lines	50

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	50
dl	0
loc	71
rs	8.6363
c	0
b	0
f	0
cc	3

How to fix Long Method

import { Inject } from '@nestjs/common';
import { CommandHandler } from '@nestjs/cqrs';
import { CreateLeaveRequestCommand } from './CreateLeaveRequestCommand';
import { ILeaveRequestRepository } from 'src/Domain/HumanResource/Leave/Repository/ILeaveRequestRepository';
import { LeaveRequest } from 'src/Domain/HumanResource/Leave/LeaveRequest.entity';
import { DoesLeaveRequestExistForPeriod } from 'src/Domain/HumanResource/Leave/Specification/DoesLeaveRequestExistForPeriod';
import { LeaveRequestAlreadyExistForThisPeriodException } from 'src/Domain/HumanResource/Leave/Exception/LeaveRequestAlreadyExistForThisPeriodException';
import { DoesLeaveExistForPeriod } from 'src/Domain/FairCalendar/Specification/DoesLeaveExistForPeriod';
import { EventsOrLeavesAlreadyExistForThisPeriodException } from 'src/Domain/FairCalendar/Exception/EventsOrLeavesAlreadyExistForThisPeriodException';
import { ICommandBus } from 'src/Application/ICommandBus';
import { CreateNotificationCommand } from 'src/Application/Notification/Command/CreateNotificationCommand';
import { NotificationType } from 'src/Domain/Notification/Notification.entity';
import { ITranslator } from 'src/Infrastructure/Translations/ITranslator';
import { ConfigService } from '@nestjs/config';
import { IDateUtils } from 'src/Application/IDateUtils';

@CommandHandler(CreateLeaveRequestCommand)
export class CreateLeaveRequestCommandHandler {
  constructor(
    @Inject('ILeaveRequestRepository')
    private readonly leaveRequestRepository: ILeaveRequestRepository,
    @Inject('ICommandBus')
    private readonly commandBus: ICommandBus,
    private readonly doesLeaveRequestExistForPeriod: DoesLeaveRequestExistForPeriod,
    private readonly doesLeaveExistForPeriod: DoesLeaveExistForPeriod,
    @Inject('ITranslator')
    private readonly translator: ITranslator,
    private readonly configService: ConfigService,
    @Inject('IDateUtils')
    private readonly dateUtils: IDateUtils
  ) {}

  public async execute(command: CreateLeaveRequestCommand): Promise<string> {
    const {
      user,
      endDate,
      endsAllDay,
      type,
      startDate,
      startsAllDay,
      comment
    } = command;

    if (
      true ===
      (await this.doesLeaveRequestExistForPeriod.isSatisfiedBy(
        user,
        startDate,
        endDate
      ))
    ) {
      throw new LeaveRequestAlreadyExistForThisPeriodException();
    }

    if (
      true ===
      (await this.doesLeaveExistForPeriod.isSatisfiedBy(
        user,
        startDate,
        endDate
      ))
    ) {
      throw new EventsOrLeavesAlreadyExistForThisPeriodException();
    }

    const leaveRequest = await this.leaveRequestRepository.save(
      new LeaveRequest(
        user,
        type,
        startDate,
        startsAllDay,
        endDate,
        endsAllDay,
        comment
      )
    );

    this.commandBus.execute(
      new CreateNotificationCommand(
        NotificationType.POST,
        this.translator.translate(
          'leave-requests-create-notification-message',
          {
            userFirstName: leaveRequest.getUser().getFirstName(),
            startDate: leaveRequest.getStartDate(),
            endDate: leaveRequest.getEndDate(),
            duration: this.dateUtils.getLeaveDuration(
              leaveRequest.getStartDate(),
              leaveRequest.isStartsAllDay(),
              leaveRequest.getEndDate(),
              leaveRequest.isEndsAllDay()
            ),
            link: `${this.configService.get<string>(
              'PERMACOOP_BASE_URL'
            )}/app/people/leaves/${leaveRequest.getId()}`
          }
        ),
        leaveRequest
      )
    );

    return leaveRequest.getId();
  }
}


1			import { Inject } from '@nestjs/common';
2			import { CommandHandler } from '@nestjs/cqrs';
3			import { CreateLeaveRequestCommand } from './CreateLeaveRequestCommand';
4			import { ILeaveRequestRepository } from 'src/Domain/HumanResource/Leave/Repository/ILeaveRequestRepository';
5			import { LeaveRequest } from 'src/Domain/HumanResource/Leave/LeaveRequest.entity';
6			import { DoesLeaveRequestExistForPeriod } from 'src/Domain/HumanResource/Leave/Specification/DoesLeaveRequestExistForPeriod';
7			import { LeaveRequestAlreadyExistForThisPeriodException } from 'src/Domain/HumanResource/Leave/Exception/LeaveRequestAlreadyExistForThisPeriodException';
8			import { DoesLeaveExistForPeriod } from 'src/Domain/FairCalendar/Specification/DoesLeaveExistForPeriod';
9			import { EventsOrLeavesAlreadyExistForThisPeriodException } from 'src/Domain/FairCalendar/Exception/EventsOrLeavesAlreadyExistForThisPeriodException';
10			import { ICommandBus } from 'src/Application/ICommandBus';
11			import { CreateNotificationCommand } from 'src/Application/Notification/Command/CreateNotificationCommand';
12			import { NotificationType } from 'src/Domain/Notification/Notification.entity';
13			import { ITranslator } from 'src/Infrastructure/Translations/ITranslator';
14			import { ConfigService } from '@nestjs/config';
15			import { IDateUtils } from 'src/Application/IDateUtils';
16
17			@CommandHandler(CreateLeaveRequestCommand)
18			export class CreateLeaveRequestCommandHandler {
19			constructor(
20			@Inject('ILeaveRequestRepository')
21			private readonly leaveRequestRepository: ILeaveRequestRepository,
22			@Inject('ICommandBus')
23			private readonly commandBus: ICommandBus,
24			private readonly doesLeaveRequestExistForPeriod: DoesLeaveRequestExistForPeriod,
25			private readonly doesLeaveExistForPeriod: DoesLeaveExistForPeriod,
26			@Inject('ITranslator')
27			private readonly translator: ITranslator,
28			private readonly configService: ConfigService,
29			@Inject('IDateUtils')
30			private readonly dateUtils: IDateUtils
31			) {}
32
33			public async execute(command: CreateLeaveRequestCommand): Promise<string> {
34			const {
35			user,
36			endDate,
37			endsAllDay,
38			type,
39			startDate,
40			startsAllDay,
41			comment
42			} = command;
43
44			if (
45			true ===
46			(await this.doesLeaveRequestExistForPeriod.isSatisfiedBy(
47			user,
48			startDate,
49			endDate
50			))
51			) {
52			throw new LeaveRequestAlreadyExistForThisPeriodException();
53			}
54
55			if (
56			true ===
57			(await this.doesLeaveExistForPeriod.isSatisfiedBy(
58			user,
59			startDate,
60			endDate
61			))
62			) {
63			throw new EventsOrLeavesAlreadyExistForThisPeriodException();
64			}
65
66			const leaveRequest = await this.leaveRequestRepository.save(
67			new LeaveRequest(
68			user,
69			type,
70			startDate,
71			startsAllDay,
72			endDate,
73			endsAllDay,
74			comment
75			)
76			);
77
78			this.commandBus.execute(
79			new CreateNotificationCommand(
80			NotificationType.POST,
81			this.translator.translate(
82			'leave-requests-create-notification-message',
83			{
84			userFirstName: leaveRequest.getUser().getFirstName(),
85			startDate: leaveRequest.getStartDate(),
86			endDate: leaveRequest.getEndDate(),
87			duration: this.dateUtils.getLeaveDuration(
88			leaveRequest.getStartDate(),
89			leaveRequest.isStartsAllDay(),
90			leaveRequest.getEndDate(),
91			leaveRequest.isEndsAllDay()
92			),
93			link: `${this.configService.get<string>(
94			'PERMACOOP_BASE_URL'
95			)}/app/people/leaves/${leaveRequest.getId()}`
96			}
97			),
98			leaveRequest
99			)
100			);
101
102			return leaveRequest.getId();
103			}
104			}
105

fairnesscoop / permacoop

Push — master ( 26b471...d6088a )

CreateLeaveRequestCommandHandler.execute B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like